Skip to content

Docs for #1165: in-progress marks only a live Run (grill, ADR 0029, glossary) - #1908

Merged
JacobStephens2 merged 11 commits into
mainfrom
1165-triage
Nov 9, 2026
Merged

JacobStephens2 merged 11 commits into
mainfrom
1165-triage

Conversation

@JacobStephens2

Copy link
Copy Markdown
Owner

Summary

The docs side of the #1165 grill, which settles that in-progress marks only a live Run. Spec #1901 and its Tickets #1902–#1904 implement it. Docs only, no code.

docs/
├── grill-with-docs/1165-claim-labels.md        # Rounds 1–3, all answered
└── adr/0029-in-progress-marks-only-a-live-run.md  # amends ADR 0026's take-over rule
GLOSSARY.md                                     # Claim, Failure limit, Ready issue, Recovery, Sweep

What the glossary now says, against main:

 Claim
-  the mark that the factory has taken an issue; stays while its pull request
-  waits for review or its Run failed, until the Day shift relabels it
+  the mark that a Run is live; ready for review → no label,
+  failure with work on origin → agent-failed in its place,
+  Spec run only waiting → ready-for-agent back on the Spec
 Failure limit
-  comes off when the Day shift … takes `in-progress` off, leaving no `needs-triage`,
-  … by relabelling a failed triage issue
+  comes off when the Day shift … takes `agent-failed` off
+  counts failed Custom processes too (#1583)
 Sweep
+  also takes `in-progress` off open issues that carry `agent-failed`
 Recovery
+  posted as a generated issue comment on `agent-failed` issues, except Security fixes;
+  a waiting Spec gets a comment naming what each Ticket waits on

Merging main brought in #1695's Triage-run glossary terms (#1894). Those are kept, except its extra take-over signal ("relabelling a failed triage issue"), which the #1165 grill (Q2.1) replaced with the single rule above. This ADR is numbered 0029 because main already has its own 0027 and 0028.

Evidence

Execution evidence

  • Before: scripts/affected-tests main, run from a stale local main without site dependencies installed: FAIL npm test --prefix site (0.17s) with ERR_MODULE_NOT_FOUND, a setup failure.
    After: scripts/affected-tests origin/main after npm ci in site/: PASS npm test --prefix site (30.53s). The selected cargo test --test share_card --test site_deploy can't build on macOS (thirdshift supports Linux only), so CI runs it.

Merge Danger

Door: two-way

Docs only. The glossary and ADR describe behaviour that #1902–#1904 haven't built yet, so until they land the glossary is ahead of the code on how a Claim ends.

Blast Radius: docs

A Spec run on #1901 reads the glossary and ADR from its Base branch, so merging this first gives it the settled vocabulary.

@JacobStephens2
JacobStephens2 merged commit 1888168 into main Nov 9, 2026
4 checks passed
@JacobStephens2
JacobStephens2 deleted the 1165-triage branch November 9, 2026 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant